Skip to content

Activate Karma Point Awards#8931

Merged
mandy-chessell merged 2 commits intoodpi:mainfrom
mandy-chessell:oak2025
Dec 20, 2025
Merged

Activate Karma Point Awards#8931
mandy-chessell merged 2 commits intoodpi:mainfrom
mandy-chessell:oak2025

Conversation

@mandy-chessell
Copy link
Copy Markdown
Contributor

Description

This PR adds an engine action to the OrganizationInsights content pack to activate the karma points awarding watchdog service to start automatically.

Related Issue(s)

In order to allow the engine action to be processed, it is necessary to allow the local repository to update its activity status. The local repositories have been updated to allow both updates and deletes of content pack elements.
This should also fix #7648.

There was a bug in the watchdog service start up in that the governance engine was initializing the listener manager in the right client and then passing the wrong client to the context - result was an NPE - all fixed.

The karma point awards service is a watch dog service. However it does not have/need a notificaiton type. I relaxed the requirement for a watchdog service to have a notification type. Alternative choices could have been to:

  • Add a notification type - however, this suggests that subscribers could be attached - what would be the value of that? Is this something we should allow?
  • Change the service to be a governance action service. Currently governance action services can listen for events. However, the karma point award service seems like a watchdog and I may move the event listening support to only be supported in the watchdog services for simplicity.

Testing

Ensuring the startup with all of the content packs active is able to record karma points for the connectors. All is good - Jacquard won :). Only stest on inmemory repo so far.

Release Notes & Documentation

This is part of the solutions update - its description will go in the organization insights solution.

Additional notes

None

mandy-chessell and others added 2 commits December 20, 2025 13:57
Signed-off-by: Mandy Chessell <mandy.e.chessell@gmail.com>
@mandy-chessell mandy-chessell merged commit 3e6a2ba into odpi:main Dec 20, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Issue deleting elements loaded by an archive

1 participant